Fix #1633: docs: update docs/architecture/workflows.md npm script table for eval commands#1655
Conversation
Add rows for `eval:lint`, `eval:run`, `eval:run:skills`, `eval:run:agents`, `eval:run:scripts`, and `eval:compare`, and note that `lint:all` now chains `eval:lint`. Closes microsoft#1633
|
@WilliamBerryiii — when you have a moment, would you take a look? No rush. I picked you based on merged 1 docs PR in last 6 months; happy to address feedback or re-route to someone better if you're not the right person. |
|
@katriendg — branch protection requires a second approval here; @WilliamBerryiii already signed off. Pinging you based on 7 merged PRs in last 6 months (active maintainer). Happy to address feedback if anything stands out, or re-route to someone better if you'd prefer. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1655 +/- ##
==========================================
+ Coverage 80.55% 85.62% +5.06%
==========================================
Files 112 86 -26
Lines 18679 12549 -6130
==========================================
- Hits 15047 10745 -4302
+ Misses 3632 1804 -1828
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
bindsi
left a comment
bindsi
left a comment
There was a problem hiding this comment.
Automated review (bindsi batch). Straightforward docs update: adds the eval:* npm script entries and corrects the lint:all description in docs/architecture/workflows.md. Accurately reflects the current package.json script surface. LGTM.
4 participants
Closes #1633
What
Adds the six new
eval:*scripts to the npm script reference table indocs/architecture/workflows.md, and updates thelint:allrow to note thateval:lintis now part of the chain.Why
These scripts were added to
package.jsonin commit 1102901 (feat(evals): add npm eval scripts and shared environments #1626) but were absent from the documentation table.Tested
Visual review of the rendered table: pipe columns align with the existing header and surrounding rows; diff is purely additive except for the one updated
lint:alldescription.Agent verification
```
branch: hunter/issue-1633
files-changed: 1
commit: 2a4911c
verification: Visual review of rendered table; pipe columns align with header and surrounding rows; diff shows only 7 additions + 1 deletion in docs/architecture/workflows.md
```